-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix lifecycle mapping metadata #69
Conversation
It appears because the 'check' is in the VERIFY phase, a package will need run so eclipse is not going to do anything past compile. I think if you test again changing that phase for the test to compile you are likely to get it working. Then the question is, do we want that in the compile phase? |
I think compile phase isn't good. At least, currently SpotBugs doesn't support incremental analysis, then analysis on compile phase should make development experience really bad. |
Any chance this fix can be finished? Looks like a test was failing against the change, but that the test needs to be updated. Would be nice to have this fix build in rather than requiring users to specify the correct lifecycle management setting themselves. |
@e-shreve-ti Would this solve your issue -> here They are supporting spotbugs now with m2e integration. |
Sorry. PEBCAK. I knew of m2e-code-quality but missed that they had multiple features and had only installed their checkstyle feature. |
@KengoTODA Can you rebase this and get working what you did? While it doesn't fix everything, if I recall it was useful and with now m2e-code-quality already working with us, we may not yet need a full solution. We probably could leverage what they have done and/or just drop this entirely and point users to their support. |
214c996
to
69f6f3a
Compare
merging, will add missing spotbugs and new verify goals separately. Also looking at m2e-code -quality to see about just introducing that code directly. |
I've fixed
<goal>
inMETA-INF/m2e/lifecycle-mapping-metadata.xml
, however this fix isn't enough to run SpotBugs from m2e. I've installed3.1.6-SNAPSHOT
in local, and confirmed that its goal is linked with lifecycle:But even when I clean the Eclipse project, it won't run SpotBugs. At least, I found nothing in in progress bar, and no report founds in
target
directory.